Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ticket 1312/new ptx out of skaled #1313

Open
wants to merge 49 commits into
base: develop
Choose a base branch
from

Conversation

sergiy-skalelabs
Copy link
Contributor

@sergiy-skalelabs sergiy-skalelabs commented Oct 26, 2022

The skale_imaTxnInsert, skale_imaTxnErase, skale_imaTxnClear, skale_imaTxnFind and skale_imaTxnListAll JSON RPC APIs are currently implemented as part of skaled and used to detect whether previous IMA Agent(s) exceeded execution of its(their) time frame(s) and still running an unfinished job. These JSON RPC APIs should be moved from skaled into IMA Agent because last one now running its own JSON RPC server.

@codecov
Copy link

codecov bot commented Oct 26, 2022

Codecov Report

Merging #1313 (5b401fb) into develop (1dd5d4e) will not change coverage.
The diff coverage is n/a.

❗ Current head 5b401fb differs from pull request most recent head 141dc93. Consider uploading reports for the commit 141dc93 to get more accurate results

@@           Coverage Diff            @@
##           develop    #1313   +/-   ##
========================================
  Coverage    89.40%   89.40%           
========================================
  Files           26       26           
  Lines         1302     1302           
  Branches       351      351           
========================================
  Hits          1164     1164           
  Misses          37       37           
  Partials       101      101           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sergiy-skalelabs sergiy-skalelabs requested a review from a team as a code owner November 1, 2022 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move IMA Agent synchronization out of skaled
2 participants